Skip to content

feat: Add wbtrace option for logger - embedding#1569

Open
dipinknair wants to merge 8 commits intomainfrom
feat/wbtrace
Open

feat: Add wbtrace option for logger - embedding#1569
dipinknair wants to merge 8 commits intomainfrom
feat/wbtrace

Conversation

@dipinknair
Copy link
Copy Markdown
Collaborator

@dipinknair dipinknair commented Mar 25, 2026 •
edited
Loading

CLoses #355

from ansys.mechanical.core.embedding.logger import Configuration
import logging
from ansys.mechanical.core import App

Configuration.configure(level=logging.ERROR, to_stdout=True, wbtracing=True)
# -- or after the fact --
Configuration.set_wbtracing(True)

app = App(version=271)
print(app)

@dipinknair dipinknair requested a review from a team as a code owner March 25, 2026 20:05
@github-actions github-actions bot added the enhancement New features or code improvements label Mar 25, 2026
@dipinknair dipinknair requested a review from koubaa March 25, 2026 20:07
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Mar 25, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 25, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 72.72727% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.28%. Comparing base (f3a3696) to head (9a2373f).

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #1569      +/-   ##
==========================================
- Coverage   80.31%   80.28%   -0.04%     
==========================================
  Files          42       42              
  Lines        3196     3206      +10     
==========================================
+ Hits         2567     2574       +7     
- Misses        629      632       +3
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 25, 2026

The documentation for this pull request will be available at https://mechanical.docs.pyansys.com/pull/1569. Please allow some time for the documentation to be deployed.

@dipinknair dipinknair requested a review from MaxJPRey April 3, 2026 20:39
@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Apr 6, 2026 •
edited
Loading

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity

Metric Results
Complexity 0

View in Codacy

🟢 Coverage 72.73% diff coverage · -0.03% coverage variation

Metric Results
Coverage variation ✅ -0.03% coverage variation (-1.00%)
Diff coverage ✅ 72.73% diff coverage

View coverage diff in Codacy

Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (f3a3696) 3196 2567 80.32%
Head commit (9a2373f) 3206 (+10) 2574 (+7) 80.29% (-0.03%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#1569) 11 8 72.73%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

TIP This summary will be updated as you push new changes. Give us feedback

@koubaa
Copy link
Copy Markdown
Collaborator

koubaa commented Apr 9, 2026

The changes are fine but I don't think it really closes #355 . For WBTRACING to be fully covered it should write to the same location as the logs (file, stdout, etc).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@klmcadams klmcadams klmcadams approved these changes

@koubaa koubaa Awaiting requested review from koubaa

@MaxJPRey MaxJPRey Awaiting requested review from MaxJPRey

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation enhancement New features or code improvements

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

WBTRACING should be covered in the logging system

4 participants

@dipinknair @koubaa @klmcadams @pyansys-ci-bot